Skip to content

[gui] migrate old Makefiles to CMake and reenable non-covered tests#21891

Closed
ferdymercury wants to merge 7 commits intoroot-project:masterfrom
ferdymercury:guifitp
Closed

[gui] migrate old Makefiles to CMake and reenable non-covered tests#21891
ferdymercury wants to merge 7 commits intoroot-project:masterfrom
ferdymercury:guifitp

Conversation

@ferdymercury
Copy link
Copy Markdown
Collaborator

@ferdymercury ferdymercury commented Apr 12, 2026

Split-off #21820 as requested by @pcanal and @linev

Depends on #21882

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 12, 2026

Test Results

    22 files      22 suites   3d 9h 6m 23s ⏱️
 3 834 tests  3 832 ✅  1 💤 1 ❌
75 673 runs  75 654 ✅ 18 💤 1 ❌

For more details on these failures, see this check.

Results for commit baeb48d.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Member

@linev linev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert changes in TFormula.

And one should try to resolve problem with libAfterImage first.
Testing of GUI classes in batch is not trivial - so enabling this test only make sense if we
really can run it without failure.

Comment thread hist/hist/src/TFormula.cxx
ferdymercury added a commit to ferdymercury/root that referenced this pull request Apr 13, 2026
@ferdymercury
Copy link
Copy Markdown
Collaborator Author

Moved to #21882 and #21901

@ferdymercury ferdymercury deleted the guifitp branch April 13, 2026 18:55
ferdymercury added a commit to ferdymercury/root that referenced this pull request Apr 14, 2026
ferdymercury added a commit to ferdymercury/root that referenced this pull request Apr 15, 2026
ferdymercury added a commit to ferdymercury/root that referenced this pull request Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants